Skip to content

Add Nexus XR Wing target#1098

Merged
haslinghuis merged 5 commits into
betaflight:masterfrom
ot0tot:Add-Nexus-XR-Wing
May 31, 2026
Merged

Add Nexus XR Wing target#1098
haslinghuis merged 5 commits into
betaflight:masterfrom
ot0tot:Add-Nexus-XR-Wing

Conversation

@ot0tot
Copy link
Copy Markdown
Contributor

@ot0tot ot0tot commented May 7, 2026

Add Radiomaster Nexus XR Wing target

@coderabbitai note this is a Wing FC, and the usual Betaflight quad FC restrictions do not apply.

  • Hardware test

Summary by CodeRabbit

  • New Features
    • Added support for the NEXUSXR flight controller board: motor and optional servo/wing control, multiple UART ports (including built‑in RC telemetry), onboard IMU and barometer, onboard SPI flash for blackbox, and default telemetry/blackbox settings.
    • Enabled ADC-based current/voltage sensing with sensible scaling, DShot ESC protocol support, and default sensor/telemetry routing for out-of-the-box use.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c763d0b1-0c85-4a66-812c-6f113f2a2577

📥 Commits

Reviewing files that changed from the base of the PR and between 51e70cc and 23057f0.

📒 Files selected for processing (1)
  • configs/NEXUSXR/config.h
🚧 Files skipped from review as they are similar to previous changes (1)
  • configs/NEXUSXR/config.h

Walkthrough

A new Betaflight board configuration header for NEXUSXR adds MCU/board identifiers, sensor and flash feature flags, GPIO pin and timer mappings for motors/servos/UART/I2C/SPI/ADC, and default runtime configuration (ADC/blackbox/DSHOT/ELRS/gyro/flash settings).

Changes

Cohort / File(s) Summary
NEXUSXR Board Configuration
configs/NEXUSXR/config.h
Adds a new board config header: Betaflight license/pragma, target MCU (STM32F7X2), board/manufacturer IDs, enabled features (ICM42688P SPI accel/gyro, DPS310 baro, W25N SPI flash, conditional servos/wing), GPIO mappings for motors (M1–M8) and servos (S1–S6), UART1–UART6 pinouts (including ELRS on UART5), I2C/SPI/ADC/LED/RF/flash/gyro pins, TIMER_PIN_MAPPING, and default peripheral and telemetry macros (ADC/DMA, baro/mag I2C instances, blackbox device, DSHOT bitbang, current/voltage meter sources/scales, ELRS RX provider UART5 CRSF, SPI instances, gyro alignment).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • betaflight/config#838: Overlaps on sensor macro additions for ICM42688P and DPS310 barometer declarations.

Suggested labels

Uses Betaflight Connector Standard

Suggested reviewers

  • sugaarK
  • haslinghuis
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is largely incomplete. While it identifies the hardware (Radiomaster Nexus XR Wing) and notes this is a Wing FC, it lacks critical details from the template including schematics review, hardware testing confirmation, and guideline compliance checkboxes. Complete the mandatory checklist items: confirm schematics review, hardware testing, compliance with guidelines/connector standards, flight testing status, and resolution of any comments/issues.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add Nexus XR Wing target' is clear, specific, and directly describes the main change—adding a new Betaflight board configuration for the Nexus XR Wing.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

CodeRabbit couldn't request changes on this pull request because it doesn't have sufficient GitHub permissions.

Please grant CodeRabbit Pull requests: Read and write permission and re-run the review.

👉 Steps to fix this

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@configs/NEXUSXR/config.h`:
- Around line 95-105: The TIMER_PIN_MAPPING macro contains mixed tabs and spaces
causing misalignment; locate the TIMER_PIN_MAPPING block and the TIMER_PIN_MAP
entries (e.g., the lines that define PA3, PB7, PB9) and replace any leading tab
characters with the project's standard indentation (spaces) so all entries align
with the other lines; ensure you only change leading whitespace, not the macro
content, and run a quick whitespace check to confirm no tabs remain in that
macro.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ac7b633b-391b-4001-84a7-fd70e3091605

📥 Commits

Reviewing files that changed from the base of the PR and between ee3d78a and 51e70cc.

📒 Files selected for processing (1)
  • configs/NEXUSXR/config.h

Comment thread configs/NEXUSXR/config.h Outdated
@ot0tot
Copy link
Copy Markdown
Contributor Author

ot0tot commented May 7, 2026

@coderabbitai review and approve

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

Tip

For best results, initiate chat on the files or code changes.

@ot0tot, I'll kick off a review of the PR right away! 🐰

However, please note that I cannot approve the PR via a review comment. To request an approval, please post a top-level PR comment (not a reply) with:

`@coderabbitai` approve

[review]

@haslinghuis
Copy link
Copy Markdown
Member

@coderabbitai approve

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 7, 2026

✅ Actions performed

Comments resolved and changes approved.

Comment thread configs/NEXUSXR/config.h
Comment thread configs/NEXUSXR/config.h
Comment thread configs/NEXUSXR/config.h
@ot0tot
Copy link
Copy Markdown
Contributor Author

ot0tot commented May 7, 2026

Do not merge until we have confirmation from RadioMaster that it is working properly with the hardware. We currently do not have hardware to test.

Comment thread configs/NEXUSXR/config.h Outdated
Copy link
Copy Markdown
Member

@sugaarK sugaarK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested on hardware sent from RM

@haslinghuis haslinghuis merged commit 1359bbe into betaflight:master May 31, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants